Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Recognize ./index.html in tar files #676

Merged
merged 1 commit into from
Jan 31, 2025
Merged

Recognize ./index.html in tar files #676

merged 1 commit into from
Jan 31, 2025

Conversation

kfranqueiro
Copy link
Contributor

@kfranqueiro kfranqueiro commented Jan 30, 2025

This updates server logic to recognize either index.html or ./index.html (and removes the superfluous negative check).

This allows archives created with commands along the lines of tar -C subdirectory/ -cf spec.tar.gz . to work.

Previously, archives needed to be created carefully, along the lines of cd subdirectory/; tar -cf ../spec.tar.gz *; cd - to prevent ./ from being prepended to all entry names.

After this update, archives with ./index.html entries work equivalently to those with index.html entries.

@kfranqueiro kfranqueiro requested a review from deniak January 30, 2025 18:36
@deniak deniak merged commit c195f82 into main Jan 31, 2025
6 of 8 checks passed
@deniak deniak deleted the kgf-dot-slash branch January 31, 2025 05:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants